home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c++-part2 / 18367 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  4.6 KB
  1. Path: ix.netcom.com!news
  2. From: miker3@ix.netcom.com (Mike Rubenstein)
  3. Newsgroups: comp.lang.c++
  4. Subject: Re: Help! What's wrong with this?
  5. Date: Fri, 19 Apr 1996 14:32:07 GMT
  6. Organization: Netcom
  7. Message-ID: <3177a34c.144208280@nntp.ix.netcom.com>
  8. References: <4l22v3$1beo@useneta1.news.prodigy.com> <317660A6.56C7@datalytics.com>
  9. NNTP-Posting-Host: ix-dc14-19.ix.netcom.com
  10. X-NETCOM-Date: Fri Apr 19  9:31:49 AM CDT 1996
  11. X-Newsreader: Forte Agent .99d/32.182
  12.  
  13. Rob Stewart <stew@datalytics.com> wrote:
  14.  
  15. > Brian Munroe wrote:
  16. > > 
  17. > > I'm trying to learn C++ and I've run into a strange problem in the
  18. > > following code.
  19. > > 
  20. > 
  21. > First, the code you submitted is C, not C++.  Just because 
  22. > you're using a C++ compiler on it does not make it C++.  Let's 
  23. > call it what it is.
  24.  
  25. OK.  Let's call it C++.  The program Brian submitted will not compile
  26. on any properly working C compiler.
  27.  
  28. > 
  29. > > int const ADD_LEN = 40;
  30. > 
  31. > Try "const int ADD_LEN = 40" instead.
  32.  
  33. Just because Brian is obviously a beginner, there's no reason to give
  34. him poor advice.  A programmer should not "try" things -- he should
  35. understand the language definition.  One who does will know that "int
  36. const ADD_LEN = 40;" and "const int ADD_LEN = 40;" are equivalent.
  37. There's no reason to change it.
  38.  
  39. > 
  40. > > int main()
  41. > > {
  42. > >   struct cust_list {
  43. > >     char name[ADD_LEN];
  44. > >     char add1[ADD_LEN];
  45. > >     char add2[ADD_LEN];
  46. > >     char add3[ADD_LEN];
  47. > >   } customer;
  48. > >   int flag;
  49. > > 
  50. > >   void get_customer_info(char temp[ADD_LEN], int flag);
  51. > 
  52. > This declaration should really come before main.
  53. > 
  54. > > 
  55. > >   for (flag=0; flag<4; flag++)
  56. > >   { switch (flag)
  57. > >     { case 0: get_customer_info(customer.name, flag); break;
  58. > >       case 1: get_customer_info(customer.add1, flag); break;
  59. > >       case 2: get_customer_info(customer.add2, flag); break;
  60. > >       case 3: get_customer_info(customer.add3, flag); break;
  61. > >       default: exit(1);
  62. > >     }
  63. > >   }
  64. > 
  65. > This is the oddest thing I've ever seen.  Don't play games like 
  66. > this, just write the following:
  67. > 
  68. >     get_customer_info(customer.name, 0);
  69. >     get_customer_info(customer.add1, 1);
  70. >     get_customer_info(customer.add2, 2);
  71. >     get_customer_info(customer.add3, 3);
  72. > 
  73. > Having said that, it would be better for you to supply the 
  74. > prompt string to get_customer_info rather than using a flag to 
  75. > do it.  This allows the caller to more appopriately associate 
  76. > the text of the prompt with the buffer being filled.
  77. > 
  78. > >   return 0;
  79. > > }
  80. > > 
  81. > > void get_customer_info(char temp[ADD_LEN], int flag)
  82. > 
  83. > FYI: Declaring temp as char[ADD_LEN] doesn't guarrantee that you 
  84. > get an array of that size.
  85. > 
  86. > > { int answer;
  87. > >   int len;
  88. > >   clrscr();
  89. > >   switch (flag)
  90. > >   { case 0:
  91. > >      cout << "Enter the bidder's name in one of the following forms.\n";
  92. > >      cout << "  Person: LAST NAME, FIRST NAME\n";
  93. > >      cout << "  Business: COMPANY NAME\n\n";
  94. > >      break;
  95. > >     case 1:
  96. > >      cout << "\nEnter first line of address\n";
  97. > >      break;
  98. > >     case 2:
  99. > >      cout << "\nEnter second line of address\n";
  100. > >      break;
  101. > >     case 3:
  102. > >      cout << "\nEnter third line of address\n";
  103. > >      break;
  104. > >     default:
  105. > >      exit(1);
  106. > >   }
  107. > >   cin >> temp;
  108. > 
  109. > Try this instead.  It will eat whitespace before filling temp.
  110. > 
  111. >     cin >> ws >> temp;
  112. > 
  113.  
  114. As opposed to
  115.  
  116.     cin >> temp;
  117.  
  118. which eats whitespace before filling temp.  The code Brian gave is
  119. correct -- why suggest a change that does nothing.
  120.  
  121. > >   len = strlen(temp);
  122. > >   if (len < ADD_LEN)
  123. > >    { strncat(temp, "                                        ",ADD_LEN-
  124. > > len);}
  125. > >   temp[ADD_LEN-1] = '\0';
  126. > 
  127. > Rather than have get_customer_info be dependent upon a global 
  128. > value, why not pass the size of temp to it.  This reduces 
  129. > dependence on the global to main, which is a better habit to 
  130. > develop (later maintenance of code is easier with restricted 
  131. > dependence on globals).
  132. > 
  133. > >   return;
  134. > > }
  135. > > 
  136. > > When I run this all 4 prompts are outputted (Is that a word?), but only
  137. > > input for the name and second line are  accepted. The input line doesn't
  138. > > accept input for the first and third lines. I'm guessing that there is a
  139. > > return character caught in the input buffer after I input the name and
  140. > > second address lines and it's read in for the first and third lines. If
  141. > > so, how do I get it out of the buffer and what the heck is it doing there
  142. > > in the first place !?
  143.  
  144. Brian's problem is that he is entering a name that contains spaces.
  145. The the first input gets the string from the first non-whitespace
  146. character to the first space and the next input gets the next string
  147. from what's already been entered.
  148.  
  149.  
  150. Michael M Rubenstein
  151.